Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add activation flow preference test #17

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

pablothedude
Copy link
Contributor

@pablothedude pablothedude commented Nov 1, 2024

Add a test tom for the preferred activation flow query params

OpenConext/Stepup-SelfService#336

Copy link
Member

@MKodde MKodde Nov 6, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome to cover these vetting nudging flows in the behat tests.

IRL you mentioned that the new feature to express the preference via a entitlement attribute was hard to test in this test suite right? OpenConext/Stepup-SelfService#336. It might be possible to collaborate with Pieter and create a new feature in the SSP SP.php test page. And let the SP notify the demo IdP to release a certain attribute if this is requested in the relay state. Just a wild idea that might be usefull in other situations.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pablothedude pablothedude force-pushed the feature/add-activation-flow-preference-test branch from 9550754 to be0acb7 Compare November 7, 2024 09:22
Add a test tom for the preferred activation flow query params
@pablothedude pablothedude force-pushed the feature/add-activation-flow-preference-test branch from be0acb7 to af5a4cd Compare November 7, 2024 10:25
I must be able to manage my second factor tokens

Scenario: A user registers a Yubikey token in selfservice using RA vetting preference
Given I receive the following attributes for "joe-a4" from the IdP:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome test!!

Suggestion: maybe create a dedicated step definition to indicate you expect the IdP to release no attributes? Perhaps a bit more expressive than the empty table?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants